Skip to content

Conversation

idegtiarenko
Copy link
Contributor

This enables a test and adds a logging around shard movements so that we could confirm it is causing issues.

Related to: #125947

@idegtiarenko idegtiarenko added >test Issues or PRs that are addressing/adding tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL v9.1.0 labels Apr 3, 2025
@idegtiarenko idegtiarenko requested a review from dnhatn April 3, 2025 09:44
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

Copy link
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

# Conflicts:
#	muted-tests.yml
@idegtiarenko idegtiarenko merged commit 5a865a4 into elastic:main Apr 4, 2025
17 checks passed
@idegtiarenko idegtiarenko deleted the 125947 branch April 4, 2025 07:17
andreidan pushed a commit to andreidan/elasticsearch that referenced this pull request Apr 9, 2025
arteam added a commit to arteam/elasticsearch that referenced this pull request Apr 14, 2025
The test was fixed and unmuted in elastic#126113, but then muted back by mistake in elastic#126193
arteam added a commit that referenced this pull request Apr 22, 2025
The test was fixed and unmuted in #126113, but then muted back by mistake in #126193
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants